[SPARK-10059] [yarn] Explicitly add JSP dependencies for tests.#8251
[SPARK-10059] [yarn] Explicitly add JSP dependencies for tests.#8251vanzin wants to merge 1 commit into
Conversation
There was a problem hiding this comment.
Does this need to be synced with the servlet API version? Ideally I suppose it does but I don't know how hard that is to plumb. I thought we had servlet 3.x in the build but you have probably looked more recently. I know JSTL is versioned differently.
There was a problem hiding this comment.
I started with that assumption, but these are actually the newest versions of those two artifacts.
There was a problem hiding this comment.
There was a problem hiding this comment.
Oh now I remember. JSP is also versioned separately such that 2.2 is part of Servlet 3.0. Carry on. http://tomcat.apache.org/whichversion.html
|
Test build #41051 has finished for PR 8251 at commit
|
|
retest this please |
|
The pyspark/yarn failure in the last run is a regression I introduced recently; I opened #8258 to fix it. |
|
Test build #41068 timed out for PR 8251 at commit |
|
YarnClusterSuite passed, so I'll merge this so that other PRs get past this point. |
Author: Marcelo Vanzin <vanzin@cloudera.com> Closes #8251 from vanzin/SPARK-10059. (cherry picked from commit ee093c8) Signed-off-by: Marcelo Vanzin <vanzin@cloudera.com>
3 participants
No description provided.